gdk: factor out utility shared among backends

Removes duplicated code and makes sure we use the version which
is safe against infinite loops

diff --git a/gdk/broadway/gdkvisual-broadway.c b/gdk/broadway/gdkvisual-broadway.c
index bad78d573a2bf5d352801f2d3cfef9737fe97185..6ea2fab05740d9d055009bc74ba65c2923ff054b 100644 (file)
--- a/gdk/broadway/gdkvisual-broadway.c
+++ b/gdk/broadway/gdkvisual-broadway.c
@@ -40,11 +40,6 @@ struct _GdkBroadwayVisualClass
   GObjectClass parent_class;
 };
 
-static void     gdk_visual_decompose_mask (gulong     mask,
-                                          gint      *shift,
-                                          gint      *prec);
-
-
 G_DEFINE_TYPE (GdkBroadwayVisual, gdk_broadway_visual, GDK_TYPE_VISUAL)
 
 static void
@@ -259,30 +254,3 @@ _gdk_broadway_screen_list_visuals (GdkScreen *screen)
 
   return list;
 }
-
-static void
-gdk_visual_decompose_mask (gulong  mask,
-                          gint   *shift,
-                          gint   *prec)
-{
-  *shift = 0;
-  *prec = 0;
-
-  if (mask == 0)
-    {
-      g_warning ("Mask is 0 in visual. Server bug ?");
-      return;
-    }
-
-  while (!(mask & 0x1))
-    {
-      (*shift)++;
-      mask >>= 1;
-    }
-
-  while (mask & 0x1)
-    {
-      (*prec)++;
-      mask >>= 1;
-    }
-}

diff --git a/gdk/gdkvisual.c b/gdk/gdkvisual.c
index 64d1ce09e922d0c4a5d4aed4ebd19cdfc4582147..3539e407c725c2d9214debd8710117d45beaa634 100644 (file)
--- a/gdk/gdkvisual.c
+++ b/gdk/gdkvisual.c
@@ -467,3 +467,30 @@ gdk_visual_get_screen (GdkVisual *visual)
 
   return visual->screen;
 }
+
+void
+gdk_visual_decompose_mask (gulong  mask,
+                           gint   *shift,
+                           gint   *prec)
+{
+  *shift = 0;
+  *prec = 0;
+
+  if (mask == 0)
+    {
+      g_warning ("Mask is 0 in visual. Server bug ?");
+      return;
+    }
+
+  while (!(mask & 0x1))
+    {
+      (*shift)++;
+      mask >>= 1;
+    }
+
+  while (mask & 0x1)
+    {
+      (*prec)++;
+      mask >>= 1;
+    }
+}

diff --git a/gdk/gdkvisualprivate.h b/gdk/gdkvisualprivate.h
index e970f550a004c4dcf5c5b2caf2b20bc60c1e71ab..3375f234de5feaa9f1c8966cc831756eeae2a91d 100644 (file)
--- a/gdk/gdkvisualprivate.h
+++ b/gdk/gdkvisualprivate.h
@@ -59,6 +59,10 @@ struct _GdkVisualClass
   GObjectClass parent_class;
 };
 
+void        gdk_visual_decompose_mask       (gulong  mask,
+                                             gint   *shift,
+                                             gint   *prec);
+
 G_END_DECLS
 
 #endif

diff --git a/gdk/quartz/gdkvisual-quartz.c b/gdk/quartz/gdkvisual-quartz.c
index ff0032b103a6757bd56331925a42afd019cae910..9f52e6ec5493c9f947f831ed1b5652b29413fe2b 100644 (file)
--- a/gdk/quartz/gdkvisual-quartz.c
+++ b/gdk/quartz/gdkvisual-quartz.c
@@ -38,27 +38,6 @@ static GdkVisual *system_visual;
 static GdkVisual *rgba_visual;
 static GdkVisual *gray_visual;
 
-static void
-gdk_visual_decompose_mask (gulong  mask,
-                          gint   *shift,
-                          gint   *prec)
-{
-  *shift = 0;
-  *prec = 0;
-
-  while (!(mask & 0x1))
-    {
-      (*shift)++;
-      mask >>= 1;
-    }
-
-  while (mask & 0x1)
-    {
-      (*prec)++;
-      mask >>= 1;
-    }
-}
-
 static GdkVisual *
 create_standard_visual (GdkScreen *screen,
                         gint       depth)

diff --git a/gdk/win32/gdkvisual-win32.c b/gdk/win32/gdkvisual-win32.c
index a2dce0537c7cc3101d84bbb1dc4562d435bf3e72..fd2a7f9455dec12eda628729558d55ab5a80e108 100644 (file)
--- a/gdk/win32/gdkvisual-win32.c
+++ b/gdk/win32/gdkvisual-win32.c
@@ -31,10 +31,6 @@
 #include "gdkprivate-win32.h"
 #include "gdkvisualprivate.h"
 
-static void  gdk_visual_decompose_mask (gulong     mask,
-                                       gint      *shift,
-                                       gint      *prec);
-
 static GdkVisual *system_visual = NULL;
 static GdkVisual *rgba_visual = NULL;
 
@@ -353,24 +349,3 @@ _gdk_win32_screen_list_visuals (GdkScreen *screen)
 
   return result;
 }
-
-static void
-gdk_visual_decompose_mask (gulong  mask,
-                          gint   *shift,
-                          gint   *prec)
-{
-  *shift = 0;
-  *prec = 0;
-
-  while (!(mask & 0x1))
-    {
-      (*shift)++;
-      mask >>= 1;
-    }
-
-  while (mask & 0x1)
-    {
-      (*prec)++;
-      mask >>= 1;
-    }
-}

diff --git a/gdk/x11/gdkvisual-x11.c b/gdk/x11/gdkvisual-x11.c
index e22c4c6679c90d926cfe4e4a37898a8dab3392cb..0614fc016e7a3e9f953aa407ed5221db3f63ca07 100644 (file)
--- a/gdk/x11/gdkvisual-x11.c
+++ b/gdk/x11/gdkvisual-x11.c
@@ -45,9 +45,6 @@ struct _GdkX11VisualClass
 };
 
 static void     gdk_visual_add            (GdkVisual *visual);
-static void     gdk_visual_decompose_mask (gulong     mask,
-                                          gint      *shift,
-                                          gint      *prec);
 static guint    gdk_visual_hash           (Visual    *key);
 static gboolean gdk_visual_equal          (Visual    *a,
                                           Visual    *b);
@@ -501,33 +498,6 @@ gdk_visual_add (GdkVisual *visual)
   g_hash_table_insert (x11_screen->visual_hash, GDK_X11_VISUAL (visual)->xvisual, visual);
 }
 
-static void
-gdk_visual_decompose_mask (gulong  mask,
-                           gint   *shift,
-                           gint   *prec)
-{
-  *shift = 0;
-  *prec = 0;
-
-  if (mask == 0)
-    {
-      g_warning ("Mask is 0 in visual. Server bug ?");
-      return;
-    }
-
-  while (!(mask & 0x1))
-    {
-      (*shift)++;
-      mask >>= 1;
-    }
-
-  while (mask & 0x1)
-    {
-      (*prec)++;
-      mask >>= 1;
-    }
-}
-
 static guint
 gdk_visual_hash (Visual *key)
 {